Skip to content

MAINT: fix docstrings for /prompt_target#1263

Merged
romanlutz merged 20 commits intomicrosoft:mainfrom
paulinek13:docs/1176/prompt_target
Dec 19, 2025
Merged

MAINT: fix docstrings for /prompt_target#1263
romanlutz merged 20 commits intomicrosoft:mainfrom
paulinek13:docs/1176/prompt_target

Conversation

@paulinek13
Copy link
Copy Markdown
Contributor

This PR is a part of the #1176 effort to improve docstring completeness and quality,
focusing on /prompt_target.

@paulinek13 paulinek13 changed the title Docs/1176/prompt target MAINT: fix docstrings for /prompt_target Dec 18, 2025
Comment thread pyrit/prompt_target/__init__.py Outdated
Comment thread pyrit/prompt_target/crucible_target.py Outdated
Comment thread pyrit/prompt_target/playwright_copilot_target.py
Comment thread pyrit/prompt_target/playwright_copilot_target.py
Comment thread pyrit/prompt_target/playwright_copilot_target.py
Comment thread pyrit/prompt_target/openai/openai_completion_target.py
Comment thread pyrit/prompt_target/openai/openai_realtime_target.py
Comment thread pyrit/prompt_target/openai/openai_target.py Outdated
Comment thread pyrit/prompt_target/openai/openai_video_target.py
Copy link
Copy Markdown
Contributor

@romanlutz romanlutz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent! Just a couple small things to fix. Thank you for the contribution @paulinek13 !

paulinek13 and others added 2 commits December 19, 2025 10:04
Co-authored-by: hannahwestra25 <hannahwestra@microsoft.com>
paulinek13 and others added 4 commits December 19, 2025 10:16
@paulinek13
Copy link
Copy Markdown
Contributor Author

@hannahwestra25 @romanlutz
Thanks for your reviews :)
I've addressed all the comments

@romanlutz romanlutz merged commit 2808205 into microsoft:main Dec 19, 2025
20 checks passed
@paulinek13 paulinek13 deleted the docs/1176/prompt_target branch January 13, 2026 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants